Skip to content

Update links to 5x api docs on use middleware page#2039

Merged
bjohansebas merged 1 commit intoexpressjs:gh-pagesfrom
ShubhamOulkar:fix
Aug 15, 2025
Merged

Update links to 5x api docs on use middleware page#2039
bjohansebas merged 1 commit intoexpressjs:gh-pagesfrom
ShubhamOulkar:fix

Conversation

@ShubhamOulkar
Copy link
Copy Markdown
Member

@ShubhamOulkar ShubhamOulkar commented Aug 14, 2025

@ShubhamOulkar ShubhamOulkar requested a review from a team as a code owner August 14, 2025 04:15
@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 14, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 5a2d55a
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/689d62dab9d5090008c05e56
😎 Deploy Preview https://deploy-preview-2039--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

🚦 Lighthouse Results (Mobile & Desktop)

URL Device Perf A11y Best Practices
/ mobile 🔴 67 🟢 100 🟢 96
/en/blog/posts.html mobile 🔴 72 🟢 96 🟢 96
/en/5x/api.html mobile 🔴 59 🟢 95 🟢 96
/ desktop 🟢 94 🟢 100 🟢 96
/en/blog/posts.html desktop 🟢 96 🟢 96 🟢 93
/en/5x/api.html desktop 🟢 95 🟢 95 🟢 96

@ShubhamOulkar
Copy link
Copy Markdown
Member Author

ShubhamOulkar commented Aug 14, 2025

I think it is better to handle dynamically update the links that are pointing to latest API docs.
So that we don't need to update on every new release.

Copy link
Copy Markdown
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@bjohansebas
Copy link
Copy Markdown
Member

I think it is better to handle dynamically update the links that are pointing to latest API docs.
So that we don't need to update on every new release.

We have this page, which contains the content of the latest API, although I’m concerned that in the future some APIs might be removed and those links could break. I’m going to open a discussion to talk about how to improve the content across versions.

https://expressjs.com/en/api.html

@bjohansebas bjohansebas merged commit b07d14a into expressjs:gh-pages Aug 15, 2025
14 checks passed
@ShubhamOulkar ShubhamOulkar deleted the fix branch August 16, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants